Skip to content

Follow-up #15546 - test for ErrObjectNotExist#15583

Merged
6543 merged 3 commits into
go-gitea:masterfrom
zeripath:fix-15257-follow-up-15546
Apr 22, 2021
Merged

Follow-up #15546 - test for ErrObjectNotExist#15583
6543 merged 3 commits into
go-gitea:masterfrom
zeripath:fix-15257-follow-up-15546

Conversation

@zeripath
Copy link
Copy Markdown
Contributor

#15546 doesn't completely fix this problem because the error returned is an ObjectNotExist
error not a BranchNotExist error.

Fix #15257

Signed-off-by: Andrew Thornton art27@cantab.net

 go-gitea#15546 doesn't completely fix this problem because the error returned is an ObjectNotExist
error not a BranchNotExist error.

Fix go-gitea#15257

Signed-off-by: Andrew Thornton <art27@cantab.net>
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Apr 22, 2021
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Apr 22, 2021
@6543
Copy link
Copy Markdown
Member

6543 commented Apr 22, 2021

🚀

@6543 6543 merged commit d6a33ce into go-gitea:master Apr 22, 2021
@zeripath zeripath deleted the fix-15257-follow-up-15546 branch April 22, 2021 17:08
zeripath added a commit to zeripath/gitea that referenced this pull request Apr 22, 2021
…exing (follow-up of go-gitea#15546) (go-gitea#15583)

Backport go-gitea#15546
Backport go-gitea#15583

 go-gitea#15546 doesn't completely fix this problem because the error returned is an ObjectNotExist
error not a BranchNotExist error.

Add test for ErrObjectNotExist too

Fix go-gitea#15257

Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added the backport/done All backports for this PR have been created label Apr 22, 2021
6543 pushed a commit that referenced this pull request Apr 22, 2021
…exing (follow-up of #15546) (#15583) (#15594)

Backport #15546
Backport #15583

 #15546 doesn't completely fix this problem because the error returned is an ObjectNotExist
error not a BranchNotExist error.

Add test for ErrObjectNotExist too

Fix #15257

Signed-off-by: Andrew Thornton <art27@cantab.net>
@go-gitea go-gitea locked and limited conversation to collaborators Jun 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[E] stats queue indexer.Index(145) failed: object does not exist [id: refs/heads/master, rel_path: ]

4 participants